Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix AF+ item names for wallpaper/carpet #81

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

atribecalledqst
Copy link

As discussed in Discord -- the Japan-only items were not in the AF+ item list. Additionally, some items that were not in AF+ (Mario carpet/wallpaper, boxing ring carpet/wallpaper, harvest carpet/wallpaper) were in the AF+ item list. Finally, correct names of old board floor / worn-out mud wall in AFe+ list.

While putting this together I learned that, at least in AF+, hex IDs past the end of the valid range can be displayed as wallpaper/carpet (though the items themselves are glitchy when dropped on the ground and in the inventory), and appear to be carpet or wallpaper either from shops or potentially even unused. Possible investigation for a follow-on PR.

Copy link
Owner

@Cuyler36 Cuyler36 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

DnM/Dongwu Senlin need their own case so that wallpapers and carpets added in later are categorized correctly.

@@ -147,7 +147,7 @@ public static ItemType GetItemType(ushort id, SaveType saveType)
return ItemType.Item;
else if (id >= 0x2C00 && id <= 0x2C5F)
return ItemType.RaffleTicket;
else if ((id >= 0x2600 && id <= 0x2642) || (id >= 0x2700 && id <= 0x2742))
else if ((id >= 0x2600 && id <= 0x263F) || (id >= 0x2700 && id <= 0x273F))
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

DnM/Dongwu Senlin need their own case so that wallpapers and carpets added in later are categorized correctly.

@erwinwallpaper
Copy link

is this solved?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants